Skip to content

added certz3.1#3360

Open
priyacj wants to merge 163 commits into
openconfig:mainfrom
priyacj:certz3.1
Open

added certz3.1#3360
priyacj wants to merge 163 commits into
openconfig:mainfrom
priyacj:certz3.1

Conversation

@priyacj

@priyacj priyacj commented Aug 3, 2024

Copy link
Copy Markdown
Contributor

certz3.1 gNSI Server Certificate Rotation

@priyacj priyacj requested a review from a team as a code owner August 3, 2024 04:08
@priyacj priyacj requested a review from a team August 3, 2024 04:08
@OpenConfigBot

OpenConfigBot commented Aug 3, 2024

Copy link
Copy Markdown

Pull Request Functional Test Report for #3360 / 8db6e2f

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
Certz-3: Server Certificate Rotation
Cisco 8000E status
Certz-3: Server Certificate Rotation
Cisco XRd status
Certz-3: Server Certificate Rotation
Juniper ncPTX status
Certz-3: Server Certificate Rotation
Nokia SR Linux status
Certz-3: Server Certificate Rotation
Openconfig Lemming status
Certz-3: Server Certificate Rotation

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
Certz-3: Server Certificate Rotation
Cisco 8808 status
Certz-3: Server Certificate Rotation
Juniper PTX10008 status
Certz-3: Server Certificate Rotation
Nokia 7250 IXR-10e status
Certz-3: Server Certificate Rotation

Help

@coveralls

coveralls commented Aug 3, 2024

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 22115486988

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 10.091%

Totals Coverage Status
Change from base Build 22113180336: 0.0%
Covered Lines: 2300
Relevant Lines: 22792

💛 - Coveralls

@morrowc morrowc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there are some Check() fialure things to address here

Comment thread feature/security/gnsi/certz/tests/internal/setup_service/setup_service.go Outdated
Comment thread feature/security/gnsi/certz/tests/internal/setup_service/setup_service.go Outdated
@dplore

dplore commented Apr 14, 2026

Copy link
Copy Markdown
Member

/fptest virtual

@dplore

dplore commented Apr 14, 2026

Copy link
Copy Markdown
Member

Thanks for all the changes @priyacj. Please share a link to a log of the test results. FYI: this is needed for every PR

@dplore

dplore commented Apr 14, 2026

Copy link
Copy Markdown
Member

@priyacj note this test fails the kne/virtual environment. At a minimum it seems like some required files are missing?
See
Log

@dplore

dplore commented Apr 14, 2026

Copy link
Copy Markdown
Member

@morrowc @mihirpitale-googler in future, please ensure test is producing expected results before approving. Thanks!

@dplore dplore left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please address comments regarding test logs and possibly missing dependencies

@dplore

dplore commented May 15, 2026

Copy link
Copy Markdown
Member

/fptest virtual

@dplore

dplore commented May 15, 2026

Copy link
Copy Markdown
Member

@priyacj note this test fails the kne/virtual environment. At a minimum it seems like some required files are missing?
See
Log

@ksgireesha

ksgireesha commented Jun 12, 2026

Copy link
Copy Markdown

@priyacj - Have you addressed, @dplore's comment above?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

juniper-pr new-requirement-pr The PR introduces a new test scenario/requirement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.